Skip to content

Fix PHP requirement in installation documentation #351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Nov 29, 2013
Merged

Fix PHP requirement in installation documentation #351

merged 4 commits into from
Nov 29, 2013

Conversation

gnutix
Copy link
Contributor

@gnutix gnutix commented Nov 17, 2013

@gregquat
Copy link
Member

Why do you want to change the version? The minimum required version is the 5.3.8 one as shown in the original documentation : https://raw.github.com/symfony/symfony-docs/master/book/installation.rst

@gnutix
Copy link
Contributor Author

gnutix commented Nov 18, 2013

This is currently discussed on the main PR I've done on the original documentation here: symfony/symfony-docs#3198

The requirement for Symfony is 5.3.3, but there are known bugs with versions lower than 5.3.8.
Whenever the main PR is accepted, I'll update this one to match it.

@gregquat
Copy link
Member

OK sorry, I didn't read the discussion. Ok, so I'll merge the PR after you have updated it. Thanks

@gnutix
Copy link
Contributor Author

gnutix commented Nov 23, 2013

@gregquat PR updated according to the main PR on english doc.

@gregquat
Copy link
Member

Thank you

gregquat added a commit that referenced this pull request Nov 29, 2013
Fix PHP requirement in installation documentation
@gregquat gregquat merged commit 84d4e3e into symfony-fr:master Nov 29, 2013
gregquat added a commit that referenced this pull request Nov 29, 2013
gregquat added a commit that referenced this pull request Nov 29, 2013
gregquat added a commit that referenced this pull request Nov 29, 2013
gregquat added a commit that referenced this pull request Nov 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants